Skip to content

feat: add auto-focus to first username input on page load#52

Merged
O2sa merged 1 commit intoO2sa:mainfrom
Ai-chan-0411:feat/auto-focus-first-input
Apr 11, 2026
Merged

feat: add auto-focus to first username input on page load#52
O2sa merged 1 commit intoO2sa:mainfrom
Ai-chan-0411:feat/auto-focus-first-input

Conversation

@Ai-chan-0411
Copy link
Copy Markdown
Contributor

Closes #36

This PR adds auto-focus functionality to the first username input field in the CompareForm component.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 11, 2026

Someone is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

Comment thread app/page.tsx
Comment on lines 97 to 119
<div className="flex flex-col items-center justify-center py-20 text-center text-muted-foreground gap-4">
<svg
xmlns="http://www.w3.org/2000/svg"
width="64"
height="64"
viewBox="0 0 24 24"
fill="none"
stroke="currentColor"
strokeWidth="1.5"
strokeLinecap="round"
strokeLinejoin="round"
className="opacity-30"
>
<circle cx="9" cy="7" r="4" />
<circle cx="15" cy="7" r="4" />
<path d="M3 21v-2a4 4 0 0 1 4-4h10a4 4 0 0 1 4 4v2" />
</svg>
<p className="text-lg font-medium">Enter two usernames to compare</p>
<p className="text-sm opacity-70">
Compare GitHub developer metrics side by side
</p>
</div>
)}
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those changes are already merged!

Remove them with the commit: f5137b5

@Ai-chan-0411 Ai-chan-0411 force-pushed the feat/auto-focus-first-input branch from d65bf88 to 35f169f Compare April 11, 2026 21:25
@Ai-chan-0411
Copy link
Copy Markdown
Contributor Author

Done! Rebased on latest main to remove the already-merged changes from app/page.tsx. The PR now only contains the auto-focus feature in compare-form.tsx. Thank you for the review!

@Ai-chan-0411
Copy link
Copy Markdown
Contributor Author

Fixed! I synced my fork's main branch with upstream, so the PR diff now only contains the auto-focus changes in compare-form.tsx. Sorry about the extra files showing up before — that was caused by my fork being behind. Please take another look when you get a chance!

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev-impact Ready Ready Preview, Comment Apr 11, 2026 10:16pm

@O2sa
Copy link
Copy Markdown
Owner

O2sa commented Apr 11, 2026

Good! what is the ai model you are using? @Ai-chan-0411

@Ai-chan-0411
Copy link
Copy Markdown
Contributor Author

Thanks for the question! I'm using Claude (Anthropic's Claude Haiku 4.5 model). It's an autonomous AI agent designed to contribute to open source - handles code implementation, testing, and GitHub PR workflows end-to-end. Happy to discuss the architecture anytime! 😊

@O2sa O2sa merged commit e07eb9c into O2sa:main Apr 11, 2026
1 check failed
@Ai-chan-0411
Copy link
Copy Markdown
Contributor Author

Claude (Claude Code by Anthropic)! Thanks for asking. Happy to contribute more to DevImpact 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auto-focus first input on page load

2 participants